Skip to content

Remove outdated phpstan ignore #596

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 25, 2025
Merged

Conversation

jdreesen
Copy link
Contributor

Seems like this is fixed now: phpstan/phpstan#8464 (comment)

@jdreesen
Copy link
Contributor Author

Ooh, the pipeline still uses PHPStan 2.1.11, and I have no idea how to update it.

On my local PHPStan 2.1.17 the error is gone.

@dbu
Copy link
Contributor

dbu commented Jun 21, 2025

thanks!

we use a phpstan github action. that has not been tagged with the latest phpstan version - i asked for it in OskarStark/phpstan-ga#85 and when this happens, i can re-run the test and it should get green.

@dbu dbu merged commit 70b52ee into FriendsOfSymfony:3.x Jun 25, 2025
17 of 18 checks passed
@dbu
Copy link
Contributor

dbu commented Jun 25, 2025

the githuab action for the phpstan github action has been fixed, so that we get latest phpstan again.

thanks for the cleanup!

@jdreesen jdreesen deleted the patch-1 branch June 25, 2025 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants